Skip to content

Add conditional SPSS transformations for Python v0.5 - #10

Draft
TonisOrmisson wants to merge 12 commits into
mainfrom
agent/conditional-transforms-v0.5
Draft

Add conditional SPSS transformations for Python v0.5#10
TonisOrmisson wants to merge 12 commits into
mainfrom
agent/conditional-transforms-v0.5

Conversation

@TonisOrmisson

@TonisOrmisson TonisOrmisson commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

What changed

  • implements Transformation Plan 0.2 conditional expressions, typed operands, and ordered schema state
  • extends the SPSS frontend with bounded COMPUTE, conditional IF, formats, measurement levels, and EXECUTE
  • applies conditional assignments in place while preserving dataset and physical table identity
  • adds canonical validation, capability metadata, documentation, and conformance coverage
  • keeps the adapter work under Unreleased, planned for Python 0.5.0

Why

Python v0.4 implements the product-neutral 0.1 plan but cannot express conditional row updates or the full 0.2 frontend contract. This branch implements that contract without adding copied datasets, staging snapshots, or an OpenStatSpec-managed history layer.

Integration dependencies

This branch is green in isolation but is not yet the final integration candidate.

  1. Specification PR Harden transformation run recovery #6 must provide and publish openstatspec-specification==0.2.1.
  2. Python lifecycle PR Recover minimal catalog and Dolt lifecycle integration #12 must land first.
  3. Specification PR Fix transformation specification fixture lookup #5 must be reconciled with specification main; then this branch must pin its final immutable commit.
  4. Rebase this branch on the lifecycle-enabled Python main, resolve overlapping API/capability/CI/catalog files, and run the complete service matrix.

The current specification commit pin is proposal-only and must not be used as a stable release claim.

Current validation

  • conditional/frontend targeted suite: 54 passed
  • full non-service suite: passed
  • GitHub Actions run #156: green after the unreleased 0.5.0 version-line correction
  • the branch remains an isolated pre-integration gate; combined lifecycle integration is still required

Readiness gate

Keep draft until the combined lifecycle + conditional branch passes Python 3.11–3.14, package smoke, PostgreSQL, MySQL, MariaDB, and Dolt fail-closed gates against the final specification commit.

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 54c9eb873c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/ci.yml Outdated

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e5a2b8d258

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/openstatspec/sql/inplace_transform.py

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a07c6b44a9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/openstatspec/transform/plan.py Outdated

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c28fd2aa3d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/openstatspec/sql/inplace_transform.py

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ee5a73e452

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tests/test_conditional_inplace_transform.py

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

Reviewed commit: d54c5298a5

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant